Skip to content

使用Rhino内置的Promise#189

Open
vtrayy wants to merge 1 commit into
aiselp:setup-v7from
vtrayy:setup-v7
Open

使用Rhino内置的Promise#189
vtrayy wants to merge 1 commit into
aiselp:setup-v7from
vtrayy:setup-v7

Conversation

@vtrayy

@vtrayy vtrayy commented Jul 14, 2026

Copy link
Copy Markdown

AI驱动,你看一下有什么问题

@codacy-production

codacy-production Bot commented Jul 14, 2026

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@aiselp

aiselp commented Jul 19, 2026

Copy link
Copy Markdown
Owner

内置的Promise不能这样简单替换,需要额外处理微任务列队

@vtrayy

vtrayy commented Jul 19, 2026

Copy link
Copy Markdown
Author

context.processMicrotasks() 已经处理了,我自己也测了,各种宏任务微任务顺序也是对的

@vtrayy

vtrayy commented Jul 19, 2026

Copy link
Copy Markdown
Author

除了协程那里需要用 let Promise = require("bluebird") 手动导入外(用 const 重声明报错),暂时没发现问题

@aiselp

aiselp commented Jul 19, 2026

Copy link
Copy Markdown
Owner

ScriptBridges.kt‎中有处理但不能覆盖所有情况,例如

importClass(java.util.concurrent.CompletableFuture);

CompletableFuture.supplyAsync(() => {
  Promise.resolve(5).then(log)
});
setTimeout(() => {}, 1000);

这里应该不会有输出

@vtrayy
vtrayy force-pushed the setup-v7 branch 4 times, most recently from 281d030 to 7388913 Compare July 19, 2026 15:32
@vtrayy

vtrayy commented Jul 20, 2026

Copy link
Copy Markdown
Author

你看一下还有什么问题

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants